Skip to content

Conversation

pushfoo
Copy link
Member

@pushfoo pushfoo commented Sep 13, 2025

Follow-up to #2766:

  • Use new SpriteCircle keyword arguments in examples (center_x, center_y)
  • Use shared kwarg dictionaries to reduce verbosity
  • Run ruff format on altered files

@pushfoo pushfoo changed the title Clean up example sprite circle kwargs Update examples to use SpriteCircle's new center_* keyword arguments Sep 13, 2025
@pushfoo pushfoo force-pushed the clean_up_example_sprite_circle_kwargs branch from 5b4b19d to 5109fb2 Compare September 13, 2025 21:21
@pushfoo
Copy link
Member Author

pushfoo commented Sep 13, 2025

CI failure's due to unpinned mypy pulling in a version with opinions on that line.

@pushfoo
Copy link
Member Author

pushfoo commented Sep 13, 2025

Fixed the mypy issue in #2768.

* Use center_x and center_y arguments for sprites

* Account for grid cells that are short and wide via min
* Use center_x and center_y arguments

* Use shared kwargs to shorten code
@pushfoo pushfoo force-pushed the clean_up_example_sprite_circle_kwargs branch from 5109fb2 to ee5a683 Compare September 13, 2025 22:00
@pushfoo pushfoo merged commit c110a4d into development Oct 1, 2025
7 checks passed
@pushfoo
Copy link
Member Author

pushfoo commented Oct 1, 2025

It's been idle for 3 weeks with no objections. We can polish further at a later time.

@pushfoo pushfoo deleted the clean_up_example_sprite_circle_kwargs branch October 1, 2025 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant